Delete non-approved revisions on user deletion

[akatsoulas]

• Delete docs without revisions on user deletion

[Copilot]

Pull Request Overview

This PR implements logic to delete documents without approved revisions during user deletion and reassign all remaining revisions to SUMO_BOT.

• Updates the user deletion handler to conditionally delete documents if they lack an approved revision and remove non‐approved revisions otherwise.
• Adds a new test case in the wiki tests to cover the behavior for non‐approved revision handling on user deletion.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
kitsune/wiki/tests/test_handlers.py	Added a test case to verify the deletion of documents without approved revisions and proper handling of non-approved revisions.
kitsune/wiki/handlers.py	Updated the on_user_deletion handler to delete documents with no approved revisions and delete or reassign non-approved revisions accordingly.

[Copilot]

If a document has multiple non-approved revisions, the loop may attempt to delete the same document multiple times, which could potentially lead to errors or unexpected behavior. Consider grouping non-approved revisions by document to ensure each document is processed only once.

[escattone]

Could this loop be replaced with the following instead?

# First delete all of the docs that have no current revision but one or more of the users' unapproved revisions.
Document.objects.filter(current_revision__isnull=True, revisions__in=non_approved_revisions).delete()
# Next, delete all of the users' unapproved revisions that were not cascade deleted above.
non_approved_revisions.delete()

[akatsoulas]

@escattone PR updated